From 083176262c8d1adb8469e637b96796bf479d1604 Mon Sep 17 00:00:00 2001
From: Ivan Diaz <diaz.it@icloud.com>
Date: Wed, 15 Oct 2025 09:14:31 -0500
Subject: [PATCH] luci-app-watchcat: use DynamicList for pinghosts

Switch to DynamicList to improve multi-host handling in the UI.

Signed-off-by: Ivan Diaz <diaz.it@icloud.com>
---
 .../luci-static/resources/view/watchcat.js       | 16 +++++++++++++---
 1 file changed, 13 insertions(+), 3 deletions(-)

diff --git a/applications/luci-app-watchcat/htdocs/luci-static/resources/view/watchcat.js b/applications/luci-app-watchcat/htdocs/luci-static/resources/view/watchcat.js
index a6c49501d2..ed9318d485 100644
--- a/applications/luci-app-watchcat/htdocs/luci-static/resources/view/watchcat.js
+++ b/applications/luci-app-watchcat/htdocs/luci-static/resources/view/watchcat.js
@@ -49,13 +49,23 @@ return view.extend({
 				1 hour would be: <b>1h</b></li><li>1 week would be: <b>7d</b></li><ul>"));
 		o.default = '6h';
 
-		o = s.taboption('general', form.Value, 'pinghosts', _('Host To Check'), _(`IP addresses or hostnames to ping.`));
-		o.datatype = 'list(host)';
-		o.default = '8.8.8.8 1.1.1.1';
+		o = s.taboption('general', form.DynamicList, 'pinghosts', _('Hosts To Check'), _(`IP addresses or hostnames to ping.`));
+		o.datatype = 'host';
+		o.default = ['8.8.8.8', '1.1.1.1'];
 		o.depends({ mode: "ping_reboot" });
 		o.depends({ mode: "restart_iface" });
 		o.depends({ mode: "run_script" });
 
+		o.load = function(section_id) {
+			return (String(this.map.data.get('watchcat', section_id, 'pinghosts') || '')
+			.trim().split(/\s+/).filter(Boolean))
+		}
+
+		o.write = function(section_id, formvalue) {
+			this.map.data.set('watchcat', section_id, 'pinghosts',
+			(formvalue || []).map(v => String(v).trim()).filter(Boolean).join(' '))
+		}
+
 		o = s.taboption('general', form.ListValue, 'addressfamily',
 				_('Address family for pinging the host'));
 		o.default = 'any';
-- 
2.30.2